-
Notifications
You must be signed in to change notification settings - Fork 18
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Sync Speedrun with last version Scaffold Stark #163
Conversation
@@ -28,7 +28,7 @@ export function getBlockExplorerTxLink(network: string, txnHash: string) { | |||
|
|||
const targetChainArr = chainNames.filter((chainName) => { | |||
const wagmiChain = chains[chainName as keyof typeof chains]; | |||
return wagmiChain.network === network; | |||
return (wagmiChain as { network: string }).network === network; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can change the import way and update the chains.Chain
to Chain
in this file
// import * as chains from "@starknet-react/chains";
import { devnet, sepolia, mainnet, Chain } from "@starknet-react/chains";
export const chains = {
devnet,
sepolia,
mainnet,
};
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great, thank you very much for the contribution.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm, smoke test looks good also new changes are present
Sync with last version Scaffold Stark
Issue #162 resolved, once the new changes are in Scaffold Stark merged (PR-355, PR-353) , I will add them and we will review them
Types of change
Comments (optional)